Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Chore]Update NOTICE.txt #38

Merged
merged 1 commit into from
Oct 24, 2024
Merged

[Chore]Update NOTICE.txt #38

merged 1 commit into from
Oct 24, 2024

Conversation

CalvinKirs
Copy link
Owner

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...

@CalvinKirs CalvinKirs merged commit afbce35 into master Oct 24, 2024
14 of 18 checks passed
Copy link

This issue is available for anyone to work on. Make sure to reference this issue in your pull request. ✨ Thank you for your contribution! ✨

Copy link

sh-checker report

To get the full details, please check in the job output.

shellcheck errors
'shellcheck ' found no issues.

shfmt errors

'shfmt ' returned error 1 finding the following formatting issues:

----------
--- env.sh.orig
+++ env.sh
@@ -163,7 +163,6 @@
     fi
     export LLVM_PROFDATA
 
-
     if [[ -z "${ENABLE_PCH}" ]]; then
         ENABLE_PCH='ON'
     fi
--- run-fe-ut.sh.orig
+++ run-fe-ut.sh
@@ -16,8 +16,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-
-
 set -eo pipefail
 
 ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")" &>/dev/null && pwd)"
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt  -w filename


Copy link

Thank you for your contribution to Apache Doris.

Don't know what should be done next? See How to process your PR

github-actions bot pushed a commit that referenced this pull request Oct 24, 2024
CalvinKirs added a commit that referenced this pull request Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant